Skip to content

Conversation

@cuviper
Copy link
Member

@cuviper cuviper commented Oct 24, 2025

Part of the release process. This PR must not be rolled up.

r? cuviper

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-release Relevant to the release subteam, which will review and decide on the PR/issue. labels Oct 24, 2025
@cuviper
Copy link
Member Author

cuviper commented Oct 24, 2025

@bors r+ p=1 rollup=never

@bors
Copy link
Collaborator

bors commented Oct 24, 2025

📌 Commit e150280 has been approved by cuviper

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 24, 2025
@bors
Copy link
Collaborator

bors commented Oct 25, 2025

⌛ Testing commit e150280 with merge e4407c0...

@bors
Copy link
Collaborator

bors commented Oct 25, 2025

☀️ Test successful - checks-actions
Approved by: cuviper
Pushing e4407c0 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Oct 25, 2025
@bors bors merged commit e4407c0 into rust-lang:master Oct 25, 2025
12 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Oct 25, 2025
@github-actions
Copy link
Contributor

What is this? This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.

Comparing f435972 (parent) -> e4407c0 (this PR)

Test differences

Show 5 test diffs

Stage 2

  • [run-make] tests/run-make/compressed-debuginfo-zstd: pass -> ignore (ignored if LLVM wasn't build with zstd for ELF section compression or LLVM is not the default codegen backend) (J0)

Additionally, 4 doctest diffs were found. These are ignored, as they are noisy.

Job group index

Test dashboard

Run

cargo run --manifest-path src/ci/citool/Cargo.toml -- \
    test-dashboard e4407c026abc5670e1213776500fac1badba7914 --output-dir test-dashboard

And then open test-dashboard/index.html in your browser to see an overview of all executed tests.

Job duration changes

  1. x86_64-rust-for-linux: 2575.0s -> 3815.4s (48.2%)
  2. x86_64-gnu-gcc: 3107.3s -> 4589.1s (47.7%)
  3. pr-check-1: 1450.5s -> 2029.0s (39.9%)
  4. arm-android: 5739.7s -> 7676.1s (33.7%)
  5. x86_64-gnu-tools: 3355.5s -> 4465.1s (33.1%)
  6. x86_64-gnu-miri: 4413.7s -> 5730.1s (29.8%)
  7. armhf-gnu: 4960.6s -> 6409.5s (29.2%)
  8. x86_64-gnu-distcheck: 6694.3s -> 8515.9s (27.2%)
  9. pr-check-2: 2373.3s -> 2971.9s (25.2%)
  10. x86_64-gnu: 6822.3s -> 8541.1s (25.2%)
How to interpret the job duration changes?

Job durations can vary a lot, based on the actual runner instance
that executed the job, system noise, invalidated caches, etc. The table above is provided
mostly for t-infra members, for simpler debugging of potential CI slow-downs.

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (e4407c0): comparison URL.

Overall result: ✅ improvements - no action needed

@rustbot label: -perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-0.3% [-0.4%, -0.3%] 6
All ❌✅ (primary) - - 0

Max RSS (memory usage)

Results (primary -0.5%, secondary 4.1%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
4.1% [4.1%, 4.1%] 1
Improvements ✅
(primary)
-0.5% [-0.5%, -0.5%] 1
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) -0.5% [-0.5%, -0.5%] 1

Cycles

This benchmark run did not return any relevant results for this metric.

Binary size

Results (secondary 0.0%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
0.0% [0.0%, 0.0%] 1
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) - - 0

Bootstrap: 474.979s -> 474.035s (-0.20%)
Artifact size: 390.46 MiB -> 390.47 MiB (0.00%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-release Relevant to the release subteam, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants